Fix list reload blink and bump to 3.2.4#74
Merged
Conversation
ShopListView and ItemTagListView flashed a full-screen LoadingView whenever they re-appeared (returning to a tab, popping back from a detail). Each .task/onChange reload sets the view-model state to .loading, which swapped already-populated content for the spinner and back — a visible blink. This makes .loading keep existing content on screen and only show LoadingView on a true cold start (.initial, or .loading with no data yet). Background refreshes now update in place. Ports the relevant subset of nativeapptemplate/NativeAppTemplate-iOS#87 — only the list views that exist in this substrate (no AccountListView here). - Blink fix in ShopListView, ItemTagListView - Bump to 3.2.4 (build 14) and update CHANGELOG - README: document the now-shared scheme (per #69) and the git update-index --skip-worktree approach for keeping a personal LAN IP out of git, replacing the stale "uncheck Shared" instructions Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Xcode upgraded the project format on open. Matches upstream NativeAppTemplate-iOS#87. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Match upstream NativeAppTemplate-iOS#87 and keep the README example value in sync. This is an example placeholder; developers override it locally and skip-worktree the scheme. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ShopListViewandItemTagListViewflashed a full-screenLoadingViewwhenever they re-appeared (returning to a tab, popping back from a detail). Each.task/onChangereload sets the view-model state to.loading, which swapped the already-populated content for the spinner and back — a visible blink.This PR makes
.loadingkeep the existing content on screen and only showsLoadingViewon a true cold start (.initial, or.loadingwith no data yet). Background refreshes (.refreshable, reappearance reloads) now update in place.Ports the relevant subset of NativeAppTemplate-iOS#87 — only the list views that exist in this substrate (the Free version has no
AccountListView).Changes
ShopListView,ItemTagListViewobjectVersion90 → 100 (Xcode upgraded the project format on open)NATIVEAPPTEMPLATE_API_DOMAINto192.168.1.21(matching upstream), keeping the README example in syncgit update-index --skip-worktreeapproach for keeping a personal LAN IP out of git, replacing the stale "uncheck Shared / keep inxcuserdata" instructionsNot ported
AccountListView— does not exist in the Free substrateTest plan
make lintpasses (0 violations, 0 formatting changes)🤖 Generated with Claude Code